gdk: Unvfuncify generic key functions

This makes Wayland and X11 no longer call into XKB and libX11 for these
functions but use GDK's own copy of these functions, just like the
win32, quartz and broadway backends.

gdk: Refactor default key vfuncs

Instead of copying them all over the place, keep a default
implementation around.

gdk: Refactor gdk_keyval_convert_case()

... so it doesn't export the function anymore.

broadway: Delete 2 files doing nothing

A function was doing nothing but calling a function that was in its own
source file doing nothing but calling a function in its own source file
that did nothing.

docs: Move property section docs to correct file

GtkSwitch: moved focus drawing from outside border inside the handle.

Merge branch 'merge-places-sidebar'

This is the new GtkPlacesSidebar widget, which is the sidebar used in
GtkFileChooser to display a list of places/folders that the user may
want to use.  File managers are encouraged to use this widget for
their own global list of folders.

Load the file chooser's widgets from the .ui file

This is Tristan's *excellent* work, minus the old code for the shortcuts bar - that is all done
in GtkPlacesSidebar now.

The UI gets loaded from a Glade resource; most of the old code to create the UI by hand is gone.

There is still code for save_widgets_create(); this needs to be moved into the UI file, but it
is not a big deal.

gtk_file_chooser_default_init() calls a new post_process_ui() that takes care of all the things
that cannot be done directly in Glade.

For now, add GtkPlacesSidebar to the Glade catalog for private widgets

Although GtkPlacesSidebar is a public widget, we can't have that in Glade's general catalog
as this branch is not merged into GTK+ proper yet.  We'll move the widget to Glade's
catalog when this is done.

Add .ui file for GtkFileChooserDefault

Updated Interlingua translation

gdk: Make atoms handled generically

This is another step towards making GdkDisplayManager backend-agnostic.

Most of the backends profit from this as their atom implementations
where generic anyway - x11 needed that to allow multiple X displays and
broadway, quartz and wayland don't have the concept of displays.

The X11 backend still did things, so I only #if 0'd some code but did
not actually update anything.

gdk: Remove unused function

displaymanager: Handle list of displays in base class

This moves the add/remove_display() functions from the subclasses to
GdkDisplay and GdkDisplayManager. It also gets rid of the list_displays
vfunc.

doc: Clarify gtk_widget_queue_draw_region() doc

displaymanager: Move set_default_display vfunc

... to GdkDisplayClass.make_default. It's only implemented by X11
anyway.

displaymanager: Handle the default display

... instead of having every backend do it on their own.

print: disconnect source_changed_handler on printeroptionwidget finalize

Printing a web page without window.print (which still segfault),
that is Ctrl+P is fixed (does not attempt to execute the
source_changed_handler on a printeroptionwidget that has been
destroyed) by disconnecting this handler in the printeroptionwidget
finalize.

https://bugzilla.gnome.org/show_bug.cgi?id=696622

gtk-demo: Fix typo

https://bugzilla.gnome.org/show_bug.cgi?id=675649

gtkprintjob: release the surface and backend before the output io channel

If the GtkPrintJob is freed too early when the surface has been created
but the job hasn't been sent to the printer, it's possible that the
file print backend tries to write to the io chaneel when it is already
closed. This produces runtime critical warnings:

GLib-CRITICAL **: g_io_channel_write_chars: assertion `channel->is_writeable' failed

https://bugzilla.gnome.org/show_bug.cgi?id=685420

gtkprintbackendfile: Fix infinite loop in _cairo_write()

It can happen if the io channel has been closed. In that case
g_io_channel_write_chars() returns early because of a g_return macro
that checks if the io channel is writable. When returning from g_return
macros, the bytes written output parameter is not updated and the error
is not filled, so the error is not detected and the written variable is
used uninitialized. We should check the return value of
g_io_channel_write_chars() to break the loop.

https://bugzilla.gnome.org/show_bug.cgi?id=685419

fix mem leak in gtk_drag_set_icon_surface, destroy pattern

https://bugzilla.gnome.org/show_bug.cgi?id=681446

Precache more atoms

Add a few more entries to the list of precached atoms.

wayland: Don't get the shell surface twice

Noticed by Pu Xingyu,
https://bugzilla.gnome.org/show_bug.cgi?id=697947

Revert "Use natural size to set adjustment ranges"

This reverts commit 666d10ec7676a15b6861b785397563cbd4ef21e6.

This change severely broke any treeviews without horizontal
scrollbars. Basically, ellipsization never kicks in, and instead
the treeview content just extends outside the visible area,
rendering it inaccessible. This broke e.g. the control-center
keyboard shortcuts panel, the gnome-disks device list, etc etc.

csd: Don't use a content_window for o-r windows

This fixes drawing of dnd icons.
https://bugzilla.gnome.org/show_bug.cgi?id=697886

wayland: Don't "probe" for a Wayland compositor if started from compositor

In the case that the client is started directly by the compositor the
WAYLAND_SOCKET environment variable is set containing the fd to use that was
created by a socketpair.

This environment variable is consumed by a call to wl_display_connect so a
second call will not take advantage of it.

https://bugzilla.gnome.org/show_bug.cgi?id=697673

Move all of GtkFileChooserDefault's fields into a private structure

I love Emacs macros, by the way.

Move GtkFileChooserDefault to gtkfilechooserdefault.c

We will move all the fields to a private structure next.

Move GtkFileChooserWidgetPrivate to gtkfilechooserwidget.c

In addition, remove the unused "file_system" field from that
structure.

Move GtkFileChooserDialogPrivate into gtkfilechooserdialog.c

We'll take private structures out of gtkfilechooserprivate.h and move
them to the appropriate place.

Remove unused #define

Don't include gtkfilechooserprivate.h from the file chooser's tests

We will make that file a fully private header; the tests have no
business poking inside the file chooser's internals at this point.

GtkPlacesSidebar: Hold a reference to the sidebar while a D-Bus proxy is being created

The creation is async, and the sidebar could be destroyed while the D-Bus proxy is being created.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Merge branch 'places-sidebar' into master

This lands the GtkPlacesSidebar widget.  It is used in
GtkFileChooserDefault, and it can also be used by third-party
applications.

Revert "GtkFileChooserDefault: Define children with a GtkBuilder template"

Having the changes for composite widget templates makes it impossible
to merge the places-sidebar branch.  So, we will merge that branch,
and *then* apply the changes for composite templates.

This reverts commit bf909f56157311174d5d9089bdec954e7def8873.

xi2: Improve pointer emulation debug reporting

Reporting "true" or "false" is nicer than the value of the flag.

https://bugzilla.gnome.org/show_bug.cgi?id=697795

gdkwindow: Handle updates created by outstanding moves in same update

If gdk_window_flush_outstanding_moves() creates new update area
we handle this directly in the same draw to avoid flashing.

This mainly affects win32 as X11 does its exposes from moves async.
However, its important for win32 since ScrollDC seems to sometimes
invalidate (and not copy) unexected regions.

http://bugzilla.gnome.org/show_bug.cgi?674051

win32: Report ScrollDC update region directly

Rather than set the window update region and repaint this region
when we get a WM_PAINT we just directly add it to the update
region. No need to roundtrip via win32.

This lets us also make sure we do this drawing in the same update
cycle. This seems especially important on Win7, because ScrollDC
seems to act kind of weird there, not using bitblt in areas where
it seemingly could, which makes scrolling look really flashy.

http://bugzilla.gnome.org/show_bug-cgi?id=674051

Make gdk_flush_outstanding_moves "reentrant"

If do_move_region_bits_on_impl causes updates they will propely
be moved by the outstanding moves we have not yet flushed.

Update gtk.symbols

This fixes win32 build

win32: Fix build

Need to take ".exe" into account for file dependencies on win32.

win32: Make build with latest gdkdisplay changes

xi2: Reset scroll valuators on synthesized crossing events

On crossing events resulting from moving windows (eg. workspace switch),
deviceid equals sourceid, so make those reset scroll valuators on all
slave devices to avoid misleading jumps in scroll events

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=690275

csd: Use symbolic icons for window buttons

With symbolic icons, we can recolor the icons as needed, and we
can use icon shadows, etc.

Don't free a list that has not been initialized

wayland: Use more explicit out parameter names

Prevent confusion between positional values and hotspot values.

Signed-off-by: Rob Bradford <rob@linux.intel.com>

Update GtkCssProvider box-shadow documentation

Non-inset box-shadows are no longer ignored by the GTK+ theming engine,
outset shadows are supported.

Signed-off-by: Rob Bradford <rob@linux.intel.com>

cds: enable outset shadows

Adds conditional code paths to GdkCssShadowValue for painting outset
shadows, and allows shadows to be applied in two passes (first outset
then inset). This can be used to draw csd shadows in outer window
borders.

https://bugzilla.gnome.org/show_bug.cgi?id=695998
Signed-off-by: Rob Bradford <rob@linux.intel.com>

wayland: don't leave root window values uninitialised

Under Wayland we don't know the absolute position of the device but there are
some API calls that expect to get an root window position. Previously we were
not assigning any value to these out parameters potentially leaving the values
undefined.

This change returns the current surface relative position of the device.

quartz: Fix call to missing symbol

This patch can be considered a leftover of commit 6f607fc8b4cd ,
replace the no longer existing function with the GDK function
meant to replace it.

This was noticeable on regular clipboard operations in quartz

window: Remember current size

Don't just look at previously remembered sizes, also look at the current
size.
This is useful for cases where the window was resized by the user or WM
and not by the application itself.

https://bugzilla.gnome.org/show_bug.cgi?id=696882

window: Split out a function

We don't want to remember sizes in the not resizable case. Also a
function named "guess_default_size" should not look at previous sizes,
it should guess.

window: Remove an unused function.

Luckily, we don't need to support an implementation of a minimal WM for
linux-fb anymore (which is what this function was added for).

docs: Fix Since tag for frame clock functions

wayland: Clean up gdkwayland.h

Make this an include-only header

window: restore size after hide/show properly

Old code assumed the size was stored in widget.allocation. This is no
longer true as the allocation is cleared upon hide. However, we store
the last configure request, and that one tracks the last size, so we can
just use that number.

Sometimes things are so easy - once you figure them out...

https://bugzilla.gnome.org/show_bug.cgi?id=696882

plug: Don't unregister windows that were never registered

Fixes warnings when trying to unregister the socket window. The socket
window is foreign, so we shouldn't register it in the first place.

https://bugzilla.gnome.org/show_bug.cgi?id=697427

build: Fix srcdir != builddir

Extract strings from .ui files without intltool

GtkVolumeButton: Define children with a GtkBuilder template

GtkFontButton: Define children with a GtkBuilder template

GtkFileChooserButton: Define children with a GtkBuilder template

GtkPathBar: Define children with a GtkBuilder template

GtkColorEditor: Define children with a GtkBuilder template

GtkColorPlane: Added construct-only properties to setup the adjustments.

This allows basic construction of this private type in GtkBuilder script,
so it can be used in the GtkColorEditor UI.

GtkPrintUnixDialog: Define children with a GtkBuilder template

GtkPageSetupUnixDialog: Define children with a GtkBuilder template

GtkRecentChooserDefault: Define children with a GtkBuilder template

GtkFontChooserDialog: Define children with a GtkBuilder template

GtkFontChooserWidget: Define children with a GtkBuilder template

GtkFileChooserDialog: Define children with a GtkBuilder template

GtkFileChooserDefault: Define children with a GtkBuilder template

GtkColorChooserDialog: Define children with a GtkBuilder template

GtkAppChooserDialog: Define children with a GtkBuilder template

GtkAppChooserWidget: Define children with a GtkBuilder template

GtkStatusbar: Define children with a GtkBuilder template

GtkScaleButton: Define children with a GtkBuilder template

GtkAssistant: Define children with a GtkBuilder template

GtkLockButton: Define children with a GtkBuilder template

GtkAboutDialog: Define children with a GtkBuilder template

GtkInfoBar: Define children with a GtkBuilder template

GtkMessageDialog: Define children with a GtkBuilder template

GtkDialog: Define children with a GtkBuilder template

As the first composite widget in GTK+, this patch also
adds some Makefile mechanics to list the ui files as
dependencies of the global GTK+ resources, and adds the
initial test case where composite classes should be tested.

Added directory with Glade catalog

This catalog can be used to work with GTK+'s private widget types,
this patch exposes a private function gtk_glade_catalog_init() which
Glade will use for the sole purpose of initializing some private widget
types in GTK+ that are referenced from various GTK+ composite widget
xml files.

Add Composite Child machinery and APIs to GtkWidget

This commit implements the needed machinery for GtkWidget
to build it's composite content from GtkBuilder XML and
adds the following API:

  o gtk_widget_init_template()

    An api to be called in instance initializers of any
    GtkWidget subclass that uses template XML to build it's components.

  o gtk_widget_class_set_template()

    API to associate GtkBuilder XML to a given GtkWidget subclass

  o gtk_widget_class_automate_child()

    API to declare an object built by GtkBuilder to be associated
    with an instance structure offset and automatically set.

 o gtk_widget_get_automated_child()

   API for bindings to fetch a child declared to be automated by
   gtk_widget_class_automate_child(), for the case where bindings
   do not generate GObjects under the hood and cannot use structure
   offsets to resolve composite object pointers.

 o gtk_widget_class_declare_callback[s]()

   Declare static functions to be used in signal callbacks from
   a given class's template XML

 o gtk_widget_class_set_connect_func()

   API for bindings to override the signal connection machinery
   for a given GtkWidget derived class.

GtkBuilder: Allow G_PARAM_CONSTRUCT properties to be set on internal children.

This patch allows properties of type G_PARAM_CONSTRUCT to be set
on internal children or explicitly constructed objects (built
with <constructor>) while previously, G_PARAM_CONSTRUCT properties
being set on already constructed objects would result in an misinformed
warning that "construct-only properties cannot be set".

G_PARAM_CONSTRUCT_ONLY properties are still refused as parameters
to already constructed children.

GtkBuilder: Add private _gtk_builder_extend_with_template()

This adds the definition of the <template> tag with some documentation
on the variant of the format.

_gtk_builder_extend_with_template() is to be used while GtkContainer
builds from composite templates. A couple of error codes are also added
to handle a few new possible failure cases.

DTD Files gtkbuilder.rnc and gtkbuilder.rng have been updated to include
the new <template> tag and it's attributes.

GtkBuilder: Added api to allow private signal callbacks

In preperation for composite objects, for better encapsulation
the following APIs are added to allow handling of signals declared
in the XML with callbacks declared statically.

  o gtk_builder_add_callback_symbol[s]()

    Adds a symbol to the internal symbol hash

  o gtk_builder_lookup_symbol()

    Looks up a symbol, exposed in case added symbols are used
    in conjunction with gtk_builder_connect_signals_full()

The default implementation of gtk_builder_connect_signals() now
does not have a strong requirement on GModule (or a strong requirement
on symbols being declared in the global namespace). Instead GModule
is used as a fallback in the case that symbols are not declared
explicitly.

Work around access to global resources without calling gtk_init()

Some utilities such as GIR and gtk-doc, initialize class vtables without
initializing GTK+, with composite templates accessing resources this
causes a hand full of unneeded warnings.

The workaround for now is the use a private function _gtk_ensure_resources()
which is both called while initializing GTK+, and at the beginning of
gtk_widget_class_set_template_from_resource() (the private function
ensures that the resource will only ever be registered GOnce).

Deprecate gtk_widget_push_composite_child & friends

Deprecate gtk_widget_push_composite_child, gtk_widget_pop_composite_child,
gtk_widget_set_composite_name, gtk_widget_get_composite_name.

This API is just bloat and was never useful, this patch deprecates
it and removes all internal calls to the composite child APIs

GtkScale: Properly chain up in GtkBuildableIface->custom_finished()

This was causing <style> markup to be ignored when specified
on GtkScale types.

wayland: Implement gdk_keymap_get_modifier_state

csd: Implement hide-titlebar-when-maximized

wayland: Fix gdk_keymap_translate_keyboard_state

I was confusing indices and masks here, which made the modifier
translation go wrong. With this commit, accelerators work.

Updated British English translation

wayland: Implement gdk_keymap_translate_keyboard_state

wayland: Fix up key event translation

The is_modifier field is supposed to be set if the key
would act as a modifier, not if any modifiers are currently
active. To fix this, introduce a private
_gdk_wayland_keymap_key_is_modifier function.

At the same time, make the hardware_keycode field in key
events actually contain the hardware keycode, not a copy
of the keyval.

trivial whitespace fix

wayland: Return NULL as the keyval name for 0

This is what the X11 backend does, and it makes the "NoSymbol"
disappear from menuitems that don't have an accelerator.